Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/permissions available2 #236

Merged
merged 6 commits into from
Jul 12, 2023
Merged

Feat/permissions available2 #236

merged 6 commits into from
Jul 12, 2023

Conversation

joelclems
Copy link
Collaborator

complète #232

@joelclems joelclems requested a review from amandine-sahl July 5, 2023 09:55
@joelclems joelclems changed the base branch from main to develop July 5, 2023 09:55
@camillemonchicourt
Copy link
Member

OK merci.
Je ne comprends pas bien comment ça fonctionne.
Il faudrait y associer un peu de documentation ?
Comment un sous-module définit les permissions disponibles qu'il propose ?
Ou alors est-ce que c'est pareil pour tous les sous-modules, ils ont tous les mêmes permissions disponibles car c'est générique et commun à tous les sous-modules de Monitoring ?

@joelclems
Copy link
Collaborator Author

joelclems commented Jul 5, 2023

Pour un module et durant l'installation, on crée les permissions disponibles pour l'object ALL et les actions CRUDE

  • si la variable permission_objects (peut être une liste contenant les objects suivants : GNM_SITES, GNM_VISITES, GNM_OBSERVATIONS, GNM_GRP_SITES) est définie dans le fichiers module.json on ajoute ces permissions disponible pour ces objets et les actions CRUDE

cf https://github.com/PnX-SI/gn_module_monitoring/blob/main/docs/sous_module.md#configuration-des-objets

@camillemonchicourt
Copy link
Member

camillemonchicourt commented Jul 7, 2023

OK, donc cela veut dire que si on donne juste un CRUVED sur le sous-module celui-ci s'applique à tous les objets (grp_sites, sites, visites, observations) ? Sauf si on définit des permissions plus fines où l'on peut avoir un CRUVED différents pour les sites et les visites par exemple ?

Peut-être à préciser/clarifier ce qui est possible de faire ou non, et comment ça fonctionne ?
Et à renseigner ici : https://github.com/PnX-SI/gn_module_monitoring/blob/develop/docs/sous_module.md#gestion-des-droits
Partie aussi à mettre à jour avec la suppression de l'héritage ?

@joelclems joelclems merged commit 0e181ff into develop Jul 12, 2023
@joelclems joelclems deleted the feat/permissions-available2 branch July 12, 2023 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants